parse.js ➔ ??? - Code Metrics - woshilapin/codingame-connector - Measure and Improve Code Quality continuously with Scrutinizer

parse.js ➔ ??? A
last analyzed 2017-07-06 20:38 UTC

↳ Parent: src/codingame/parse.js

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
nc	4
nop	1
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0

/**
 * @file Module 'codingame/parse'
 * @author woshilapin <[email protected]>
 */
/**
 * Parse response coming from Codingame's services
 * @module codingame/parse
 */
import CodingameError from './error.js';
import parse_frames from './parsers/frames.js';
import parse_failcompile from './parsers/failcompile.js';
import parse_failexpected from './parsers/failexpected.js';
import parse_success from './parsers/success.js';

let parsers = [];
parsers.push(parse_frames);
parsers.push(parse_failcompile);
parsers.push(parse_failexpected);
parsers.push(parse_success);

/**
 * Parse response coming from tests sent to Codingame website
 *
 * @name parse
 * @function
 * @param {Object} response Body of the response
 * @returns {Promise<Object|CodingameError>} Resolve if test is a success, reject with a CodingameError if test failed or parsing was impossible
 */
let parse = (response) => {
	let body = response.success;
	for (let parser of parsers) {
		try {
			let promise = parser.parse(body);
			return Promise.resolve(promise); // Resume loop if no Error has been thrown
		} catch (error) {
			continue;

		}
	}
	let message = `Unknown error. Codingame may have changed its API.  Please contact 'codingame-connector' developer.`;
	let error = new Error(message);

	return Promise.reject(error);
};

export default {
	"parse": parse
};


1			/**
2			* @file Module 'codingame/parse'
3			* @author woshilapin <[email protected]>
4			*/
5			/**
6			* Parse response coming from Codingame's services
7			* @module codingame/parse
8			*/
9			import CodingameError from './error.js';
10			import parse_frames from './parsers/frames.js';
11			import parse_failcompile from './parsers/failcompile.js';
12			import parse_failexpected from './parsers/failexpected.js';
13			import parse_success from './parsers/success.js';
14
15			let parsers = [];
16			parsers.push(parse_frames);
17			parsers.push(parse_failcompile);
18			parsers.push(parse_failexpected);
19			parsers.push(parse_success);
20
21			/**
22			* Parse response coming from tests sent to Codingame website
23			*
24			* @name parse
25			* @function
26			* @param {Object} response Body of the response
27			* @returns {Promise<Object\|CodingameError>} Resolve if test is a success, reject with a CodingameError if test failed or parsing was impossible
28			*/
29			let parse = (response) => {
30			let body = response.success;
31			for (let parser of parsers) {
32			try {
33			let promise = parser.parse(body);
34			return Promise.resolve(promise); // Resume loop if no Error has been thrown
35			} catch (error) {
36			continue;
			0 ignored issues – show Unused Code introduced 2016-11-09 19:18 UTC by Report Bug Copy Issue Report This continue has no effect on the loop flow and can be removed. Loading history...
37			}
38			}
39			let message = `Unknown error. Codingame may have changed its API. Please contact 'codingame-connector' developer.`;
40			let error = new Error(message);
			0 ignored issues – show Comprehensibility introduced 2016-11-09 19:18 UTC by Report Bug Copy Issue Report You are assigning a value to the catch variable `error`, thereby shadowing the original value. Loading history... Comprehensibility Naming Best Practice introduced 2016-11-09 19:18 UTC by Report Bug Copy Issue Report The variable `error` already seems to be declared on line `35`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
41			return Promise.reject(error);
42			};
43
44			export default {
45			"parse": parse
46			};
47

woshilapin / codingame-connector

parse.js ➔ ??? A last analyzed 2017-07-06 20:38 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

parse.js ➔ ??? A
last analyzed 2017-07-06 20:38 UTC